app/components/PaginationControls.vue (1)

5-5: Rename the threshold constant before the typo spreads further.

ALL_PAGES_VISIBLE_TRESHOLD is misspelt, which makes the intent look accidental and makes future searches/reuse harder. Please rename it to ALL_PAGES_VISIBLE_THRESHOLD while it is still local to this component.

As per coding guidelines, **/*.{ts,tsx,vue} should "Use clear, descriptive variable and function names".

Also applies to: 68-68, 104-104

app/composables/useSettings.ts (1)

5-5: Prefer the shared-types auto-import here.

app/composables/* already gets shared/types/* exports auto-imported, so this extra type import is unnecessary noise.

Based on learnings, exports from shared/types/* are auto-imported by Nuxt for composables and components, so explicit imports for those types are unnecessary.

i18n/locales/en.json (1)

75-76: Don't bake the cap values into the English copy.

Line 76 hard-codes 1,000 and 5,000, but the real caps already live in the search logic. If those limits change again, the UI text will drift even though the behaviour is correct. Passing the caps as i18n params would keep the message aligned with enforcement.

app/pages/search.vue (1)

455-474: Verify destructuring in watch callback behaves as intended.

The pattern watch(displayResults, ([firstResult]) => { ... }) destructures the first element from the watched array. This works because Vue's watch callback receives the new value (the array) as its first argument, and array destructuring extracts the first element.

However, this pattern may be unfamiliar to some developers. Consider adding a brief comment or using the more explicit form for clarity:

Optional: More explicit alternative

-watch(displayResults, ([firstResult]) => {
+watch(displayResults, (results) => {
+  const firstResult = results[0]
   if (!pendingEnterQuery.value) return

The current implementation is functionally correct and type-safe (thanks to optional chaining at line 470), so this is purely a readability suggestion.